Skip to content

Migrate rust related PR from apache/nuttx-app #90

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

fxdmhtt
Copy link

@fxdmhtt fxdmhtt commented Jul 11, 2025

Summary

Sync the following PRs:
apache/nuttx-apps#3020
apache/nuttx-apps#2980
apache/nuttx-apps#2967
apache/nuttx-apps#2957
apache/nuttx-apps#2955
apache/nuttx-apps#2944
apache/nuttx-apps#2487

Impact

none

Testing

It has been tested on apache/nuttx-apps. I have also tested it on open-vela.

@CLAassistant
Copy link

CLAassistant commented Jul 11, 2025

CLA assistant check
All committers have signed the CLA.

@xiaoxiang781216
Copy link
Collaborator

need keep the individual patch as it instead merging into one.

@fxdmhtt
Copy link
Author

fxdmhtt commented Jul 11, 2025

I don't quite understand why we need to keep a individual patch? These 7 PRs are actually repeated improvements on these files, and they should essentially be 7 commits under one PR.
Is there a simple way to migrate these 7 PRs?

@xiaoxiang781216
Copy link
Collaborator

I don't quite understand why we need to keep a individual patch? These 7 PRs are actually repeated improvements on these files, and they should essentially be 7 commits under one PR. Is there a simple way to migrate these 7 PRs?

I mean you should keep the original patch as it instead merging into two patch:
apache/nuttx-apps#3020
apache/nuttx-apps#2980
apache/nuttx-apps#2967
apache/nuttx-apps#2957
apache/nuttx-apps#2955
apache/nuttx-apps#2944
apache/nuttx-apps#2487

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants